Skip to content

Use collection expression for ASP.NET Core Web API project template's WeatherForecastController.cs #58194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 1, 2024
Merged

Use collection expression for ASP.NET Core Web API project template's WeatherForecastController.cs #58194

merged 1 commit into from
Oct 1, 2024

Conversation

@tikap
Copy link
Contributor

@tikap tikap commented Oct 1, 2024

Use collection expression for ASP.NET Core Web API project template's WeatherForecastController.cs

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Use collection expressions syntax to create string array.

Description

For the "ASP.NET Core Web API" project template (used by Visual Studios), use the new collection expressions introduced in .NET 8.0 to create the "Summaries" string array to prevent styling warnings, i.e ide0300.

Related Issue: #58175

@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Oct 1, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Oct 1, 2024
captainsafia
captainsafia approved these changes Oct 1, 2024
View reviewed changes
Copy link
Member

@captainsafia captainsafia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks you!

@captainsafia captainsafia enabled auto-merge (squash) October 1, 2024 20:54
@captainsafia captainsafia merged commit 403a72d into dotnet:main Oct 1, 2024
27 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview1 milestone Oct 1, 2024
captainsafia pushed a commit that referenced this pull request Dec 31, 2024
@tikap @captainsafia
Use collection expression to create Summaries variable in WeatherFore…
9c40ad3 
…castController (#58194)
captainsafia pushed a commit that referenced this pull request Feb 11, 2025
@tikap @captainsafia
Use collection expression to create Summaries variable in WeatherFore…
eccb20a 
…castController (#58194)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@captainsafia captainsafia captainsafia approved these changes

Assignees

No one assigned

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Milestone

10.0-preview1

Development

Successfully merging this pull request may close these issues.

2 participants

@tikap @captainsafia